Skip to content

Conversation

@profxj
Copy link
Contributor

@profxj profxj commented Jan 1, 2026

Docs galore, thanks to Claude

You can see how they are rendering here:

https://frb-zdm.readthedocs.io/en/docs/

@cwjames1983
Copy link
Collaborator

@profxj This is amazing! Nice work @claude. The only "features" I'd request are to A: write somewhere how to "make" (or otherwise view) the docs - even a simple link to https://frb-zdm.readthedocs.io/en/docs/ , and B: a short acknowledgement somewhere that the doc strings were generated primarily using Claude.

@cwjames1983
Copy link
Collaborator

Also - what would you tell yourself about making useful comments for each commit? ;-)

Copy link
Collaborator

@cwjames1983 cwjames1983 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As per my comment: This is amazing! Nice work @claude. The only "features" I'd request are to A: write somewhere how to "make" (or otherwise view) the docs - even a simple link to https://frb-zdm.readthedocs.io/en/docs/ , and B: a short acknowledgement somewhere that the doc strings were generated primarily using Claude.

But I'm happy to approve this as of now, and make the above optional extras

@cwjames1983 cwjames1983 merged commit fd595a9 into main Jan 8, 2026
6 checks passed
@cwjames1983 cwjames1983 deleted the docs branch January 8, 2026 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants